PG-2238 - Add the OIDC topic#930
Conversation
This PR reorganizes the authentication methods in Solutions, adding a folder to future proof the authentication methods, and adds the OIDC topic.
- add content to oidc, which includes beginner explanations for authentication flow, why use OIDC and standard setup from the PG side.
- Add buttons to overview, and add a lot of links to the oidc chapter for all of our sources.
- Proper installation steps, updated step 1 with readme info and remove step 2 - Reordered TOC to put OIDC first - create a snippet folder with proper oidc authentication flow which can be modified any time to render immediately as opposed to having an svg, using mermaid. - standardize the external identity providers, making a list of them is too bloaty.
dutow
left a comment
There was a problem hiding this comment.
Is the oidc-auth-flow.svg still needed?
- Removed important note at the end as it's redundant - Cleaned step 1 to make it clearer how users can install the package which is already bundled - Rewrote examples in step 3 to be clearer
This question is still open, that seems to be a leftover file to me, we don't need it now with mermaid. |
I believe we can keep it, it's legacy content and it's good to have it for historical reference even though we don't use it anymore. |
It is a newly added file in this PR? I don't see how that's historical context (or why we need historical context outside the git history) |
Yep basically git history. But I'll cut it out now. |
This PR reorganizes the authentication methods in Solutions, adding a folder to future proof the authentication methods, and adds the OIDC topic.